Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose workflow job metrics via new actions-metrics-server #2057

Merged
merged 8 commits into from
Dec 9, 2022

Conversation

mumoshu
Copy link
Collaborator

@mumoshu mumoshu commented Nov 30, 2022

This is an enhanced version of @ColinHeathman's #1814, which introduces a dedicated command/application for receiving workflow job webhook events and exposing Prometheus metrics, so that you can scale the github-webhook-server and the new actions-metrics-server separately and independently.

This also updates the Helm chart to accept new values under actionsMetrics and actionsMetricsServer for configuring ServiceMonitor and a set of K8s resources for the actions-metrics-server separately.

I'm submitting this as a new pull request, because for whatever reason I was unable to push/force-push to the source branch of #1814. (Perhaps it's due to that the branch is under the author's GitHub organization, not the author's account?

@mumoshu mumoshu changed the title Re: Add workflow job metrics to Github webhook server Expose workflow job metrics via the new actions-metrics-server Dec 7, 2022
@mumoshu mumoshu changed the title Expose workflow job metrics via the new actions-metrics-server Expose workflow job metrics via new actions-metrics-server Dec 7, 2022
@mumoshu mumoshu added this to the v0.27.0 milestone Dec 7, 2022
@mumoshu
Copy link
Collaborator Author

mumoshu commented Dec 9, 2022

@toast-gear Thanks a lot for reviewing! ☺️

@cep21
Copy link

cep21 commented Jul 18, 2023

Hi @mumoshu , any thoughts on the bug here and proposed fix: (#2743). I believe it's related to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants